Skip to content
This repository has been archived by the owner on Oct 8, 2020. It is now read-only.

Replaced vague warning about concurrent memberships (with the same organization) with more detailed information and link. #250

Merged
merged 2 commits into from Feb 24, 2018

Conversation

universalhandle
Copy link
Contributor

No description provided.

…ganization) with more detailed information and link.
@seancolsen
Copy link
Contributor

Thanks @GinkgoFJG

I made a minor adjustment to the syntax of the URLs in order to conform to our internal url standards.

@seancolsen seancolsen merged commit 12ed458 into civicrm:master Feb 24, 2018
@universalhandle
Copy link
Contributor Author

Thanks, @seanmadsen. You know, as I was writing the URLs, I thought, "This can't be right." I referenced several other pages with internal links and they were using absolute URLs, so I went with it.

For what it's worth, the internal URL standards link is not easily discoverable for new (newish?) contributors. From the main GitHub page for the user guide, I followed the link under the "Contributing to this guide" header to the Contributing to this guide page and then the Documentation style guide, which has several headings relating to URLs, but none of which answered my question. I don't remember whether or not it occurred to me to look in the left menu for related content in the same chapter, but I'm sure I wouldn't have thought to look in the chapter named Markdown.

Does my brain just work different than most, or do you think there's a way we can make this easier to find for the next person?

@seancolsen
Copy link
Contributor

Great feedback, @GinkgoFJG and I agree very much. Thanks for the specifics. I'll work on improving that!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants